-
Notifications
You must be signed in to change notification settings - Fork 186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Clifford-only simulator (Stim) #2193
Conversation
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These changes look great! Thanks, Ben.
I don't see any new tests... It seems all of the C++ tests will be run except the ones which are skipped with #ifndef CUDAQ_BACKEND_STIM
. What about Python - may be we should add a couple which showcase which gates are supported and behavior on unsupported gates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
* Add Python tests (with negative tests) * Throw an addition error message * Add TODO
Thanks for the suggestion, Pradnya. I added some Stim-specific backend tests in 20343fd. Please take a look and let me know if that's what you had in mind, or if you'd like to see something else, too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
* Reduce the number of files built to only those needed by libstim. This makes the build go a little bit faster. * Remove warnings when compiling libstim to not clutter up console * Make the warning custimizations ONLY applicable to Stim submodule building (i.e. allow regular warnings/errors in StimCircuitSimulator.cpp) * Remove unneeded macro
CUDA Quantum Docs Bot: A preview of the documentation can be found here. |
Add https://github.com/quantumlib/Stim (v1.13.0) simulator for Clifford-only circuits.
The user requests it like other targets:
--target stim
andcudaq.set_target('stim')
.